Skip to content

Conversation

@katinthehatsite
Copy link
Contributor

Related issues

Fixes STU-702

Proposed Changes

This PR adds the error handling for when there is an error fetching remote children in the file free:

Screenshot 2025-12-17 at 11 26 44 AM

Testing Instructions

  • Pull the changes from this branch
  • Apply the following diff:
index 1ffafad9..66c37d1c 100644
--- a/src/stores/sync/sync-api.ts
+++ b/src/stores/sync/sync-api.ts
@@ -105,6 +105,9 @@ export const fetchRemoteFileTree = createAsyncThunk(
                path: string;
                parentChecked?: boolean;
        } ) => {
+               // TEMPORARY: Simulate error for testing remote file tree
+               throw new Error( 'Simulated remote file tree error for testing' );
+               
                const requestBody: BackupLsRequest = {
                        backup_id: rewindId,
                        path,
  • Start Studio with npm start
  • Ensure that you have at least one WP.com site connected to a local Studio site
  • Navigate to the Sync tab
  • Click on the Pull button
  • Open the Specific files and folders section
  • Confirm that you can see an error from the screenshot when you try to open wp-content

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@katinthehatsite katinthehatsite self-assigned this Dec 17, 2025
@katinthehatsite katinthehatsite marked this pull request as draft December 17, 2025 10:57
@katinthehatsite
Copy link
Contributor Author

I paused this for now to wait until we agree in #2271 if the design is fine and if we want a retry. After that, I will adjust this PR as it follows the approach that is similar.

Base automatically changed from fix/add-error-node-for-fetching-local-tree to trunk December 22, 2025 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants